Skip to content

Comments

Fix: Comprehensive Fixes and Enhancements for User Feedback System#205

Merged
Annanyatiwary4 merged 2 commits intoDevSyncx:mainfrom
kmanish527:bug/feedback
Oct 29, 2025
Merged

Fix: Comprehensive Fixes and Enhancements for User Feedback System#205
Annanyatiwary4 merged 2 commits intoDevSyncx:mainfrom
kmanish527:bug/feedback

Conversation

@kmanish527
Copy link
Contributor

🚀 Pull Request Checklist

  • ✅ My issue is assigned to me, and I have not taken up another task simultaneously.
  • 🔁 I have pulled the latest changes from the main branch.
  • 🧪 My code is tested and does not break existing functionality.
  • 📚 I have added/updated documentation wherever necessary.
  • 🧹 My code follows the project’s coding standards.
  • ✍️ My commits are clear and meaningful.
  • 🧾 I have linked the issue this PR addresses with Closes #204 .

📌 Related Issue

Closes #204

🧠 Description

  • A critical backend bug was resolved where a user's userId was being saved even on anonymous submissions. This was fixed by making the userId optional in the database model and updating the controller to conditionally add the ID only if the submission is not anonymous.
image
  • A frontend bug was fixed where the feedback popup would display stale data from the previous submission. The component's state is now programmatically reset every time the modal is opened, ensuring a clean, blank form for the user.
image
  • All native browser alert()s were replaced with modern, non-blocking react-toastify notifications. This logic was centralized within the Feedback to handle success and error states, providing a more professional and seamless user experience.
image

✅ Type of Change

  • 🐞 Bug fix
  • 💡 Feature
  • 🧹 Code cleanup/refactor
  • 🧪 Test cases
  • 📚 Docs update

@vercel
Copy link

vercel bot commented Oct 29, 2025

Someone is attempting to deploy a commit to the Annanya's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

🙌 Thanks @kmanish527 for raising this PR!
Our team will review it soon ⏳ — meanwhile, please make sure your PR follows our contributing guidelines 👉 Contributing Guidelines 🎉

@Annanyatiwary4 Annanyatiwary4 merged commit 465bb29 into DevSyncx:main Oct 29, 2025
1 check failed
@github-actions
Copy link

🎉🎉 Thank you for your contribution! Your PR #205 has been merged! 🎉🎉

@Annanyatiwary4 Annanyatiwary4 added enhancement New feature or request Frontend backend hacktoberfest-accepted only for hacktoberfest 2025 candidate labels Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend enhancement New feature or request Frontend hacktoberfest-accepted only for hacktoberfest 2025 candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: userId is saved to the database even when feedback is submitted anonymously

2 participants